Update Emscripten build to llvm 22#907

Merged

mcbarton merged 24 commits intocompiler-research:mainfrom

mcbarton:emscripten-llvm22

Apr 26, 2026

Collaborator

mcbarton commented Apr 22, 2026

No description provided.

mcbarton marked this pull request as draft

April 22, 2026 08:47

mcbarton mentioned this pull request

[Feature Request]: Add LLVM 22 support #871

Open

8 tasks

codecov Bot commented Apr 22, 2026 •

edited

Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.87%. Comparing base (cbef716) to head (c1714df).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #907   +/-   ##
=======================================
  Coverage   80.87%   80.87%           
=======================================
  Files          15       15           
  Lines        4707     4707           
=======================================
  Hits         3807     3807           
  Misses        900      900

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

mcbarton force-pushed the emscripten-llvm22 branch 3 times, most recently from 3887f62 to 5e14c06 Compare

April 22, 2026 10:12

mcbarton added 2 commits

April 22, 2026 12:21


          Update Emscripten llvm support to llvm 22

4c692a4


          Update exports.ld

ddbb2ac

mcbarton force-pushed the emscripten-llvm22 branch from f2ae833 to ddbb2ac Compare

April 22, 2026 11:21

Collaborator Author

mcbarton commented Apr 22, 2026

The current failures for the Emscripten llvm 22 based build

1: [  FAILED  ] CppInterOpTest/InProcessJIT.FunctionReflection_JitCallAdvanced, where TypeParam = <type>
1: [  FAILED  ] CppInterOpTest/InProcessJIT.FunctionReflection_ConstructPOD, where TypeParam = <type>
3: [  FAILED  ] TracingTest.JitCallWrapperSourceLogged

Collaborator Author

mcbarton commented Apr 22, 2026

@anutosh491 do you see anything obvious from the failures, as to what might have regressed on the llvm side to make these tests fail? Looking at least 2 of the tests, they failed for older llvms, so I assume you did some fix in the past to make them work, and that might have regressed.

anutosh491 reviewed

View reviewed changes

Collaborator

anutosh491 left a comment •

edited

Loading

I don't remember sadly if something was done to get the above tests working :(

But that being said I think I know how to fix the patches upstream. I have something I mind I need to find the time to try out.

I'll try something this weekend !

mcbarton added 4 commits

April 25, 2026 10:54


          Merge branch 'main' into emscripten-llvm22

d9c6ff1


          Update action.yml

7ab6c0e


          Disable failing tests for Emscripten llvm 22 build

5075fc0


          Fix previous commit

8e847e9

mcbarton marked this pull request as ready for review

April 25, 2026 10:23

mcbarton changed the title ~~[wip] Update Emscripten build to llvm 22~~ Update Emscripten build to llvm 22

mcbarton added 2 commits

April 25, 2026 12:15


          Update FunctionReflectionTest.cpp

79ffc47


          Update TracingTests.cpp

804a86f

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"

mcbarton added 2 commits

April 25, 2026 12:54


          Update FunctionReflectionTest.cpp

2be4fc0


          Update TracingTests.cpp

f9f21f7

github-actions Bot reviewed

View reviewed changes

Contributor

github-actions Bot left a comment

clang-tidy made some suggestions

unittests/CppInterOp/FunctionReflectionTest.cpp

unittests/CppInterOp/FunctionReflectionTest.cpp

unittests/CppInterOp/TracingTests.cpp Outdated

mcbarton added 2 commits

April 25, 2026 13:14


          Update FunctionReflectionTest.cpp

8df3d89


          Update TracingTests.cpp

1e1bc07

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"

mcbarton and others added 3 commits

April 25, 2026 15:38


          Merge branch 'main' into emscripten-llvm22

9a23965


          Try to fix

941a9aa


          Update emscripten.yml

295020e

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"


          Update action.yml

0d483c6

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"


          Merge branch 'main' into emscripten-llvm22

b0f408d

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"


          Merge branch 'main' into emscripten-llvm22

cf4bb1b

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"


          Update action.yml

8fe21ea

Contributor

github-actions Bot commented Apr 25, 2026

clang-tidy review says "All clean, LGTM! 👍"

mcbarton added 2 commits

April 26, 2026 09:24


          Try fix

356eed0


          Merge branch 'main' into emscripten-llvm22

dd3368e

github-actions Bot reviewed

View reviewed changes

Contributor

github-actions Bot left a comment

clang-tidy made some suggestions

unittests/CppInterOp/TracingTests.cpp


          Merge branch 'main' into emscripten-llvm22

c1714df

mcbarton requested a review from anutosh491

April 26, 2026 10:13


          Merge branch 'main' into emscripten-llvm22

4c44d26

vgvassilev approved these changes

View reviewed changes

Contributor

vgvassilev left a comment

Lgtm, provided we have our caches properly rebuilt soon…

anutosh491 approved these changes

View reviewed changes

Collaborator

anutosh491 left a comment

The changes look good !


          Fix clang-runtime deployment

882bad2

mcbarton merged commit 2626ddd into compiler-research:main

9 of 26 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet